Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Tests] appveyor -> GHA (run tests on Windows in both pwsh and WSL + Ubuntu) #2987

Merged
merged 1 commit into from
Mar 22, 2024
Merged

[Tests] appveyor -> GHA (run tests on Windows in both pwsh and WSL + Ubuntu) #2987

merged 1 commit into from
Mar 22, 2024

Conversation

joeyguerra
Copy link
Contributor

Converting the appveyor pipeline to GitHub Actions.

.eslintrc Outdated Show resolved Hide resolved
.github/workflows/native-wsl.yml Outdated Show resolved Hide resolved
.github/workflows/native-wsl.yml Outdated Show resolved Hide resolved
.github/workflows/native-wsl.yml Outdated Show resolved Hide resolved
.nycrc Outdated Show resolved Hide resolved
tests/src/core/resolve.js Outdated Show resolved Hide resolved
.github/workflows/native-wsl.yml Outdated Show resolved Hide resolved
.eslintrc Outdated Show resolved Hide resolved
@joeyguerra
Copy link
Contributor Author

I've confirmed that separate runs using Vampire/setup-wsl are disconnected in that you can't use npm in one run when Node is installed in another one. In addition, nvm is unavailable in a run when using ljharb/actions/node/install in a previous run. Thus necessitating the whole script be done in 1 run for the WSL configuration.

I've also taken a different strategy to address the other problems (typically unexpected token errors). Which results in passing tests without having to ignore checks in the utils/ code.

@ljharb would you mind taking another look at this PR?

Copy link
Member

@ljharb ljharb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, if the tests pass then I'm happy with this approach - I can consolidate things into reusable scripts later :-)

.github/workflows/native-wsl.yml Outdated Show resolved Hide resolved
.github/workflows/native-wsl.yml Outdated Show resolved Hide resolved
.github/workflows/native-wsl.yml Outdated Show resolved Hide resolved
.github/workflows/native-wsl.yml Outdated Show resolved Hide resolved
.github/workflows/native-wsl.yml Outdated Show resolved Hide resolved
.github/workflows/native-wsl.yml Outdated Show resolved Hide resolved
.github/workflows/native-wsl.yml Outdated Show resolved Hide resolved
.github/workflows/native-wsl.yml Show resolved Hide resolved
@ljharb ljharb changed the title chore: Run tests on Windows in both pwsh and WSL (Ubuntu) [Tests] appveyor -> GHA (run tests on Windows in both pwsh and WSL + Ubuntu) Mar 22, 2024
@ljharb ljharb merged commit fa60e3d into import-js:main Mar 22, 2024
192 of 193 checks passed
renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Sep 4, 2024
| datasource | package              | from   | to     |
| ---------- | -------------------- | ------ | ------ |
| npm        | eslint-plugin-import | 2.29.1 | 2.30.0 |


## [v2.30.0](https://github.com/import-js/eslint-plugin-import/blob/HEAD/CHANGELOG.md#2300---2024-09-02)

##### Added

-   \[`dynamic-import-chunkname`]: add `allowEmpty` option to allow empty leading comments (\[[#2942](import-js/eslint-plugin-import#2942)], thanks \[[@JiangWeixian](https://github.com/JiangWeixian)])
-   \[`dynamic-import-chunkname`]: Allow empty chunk name when webpackMode: 'eager' is set; add suggestions to remove name in eager mode (\[[#3004](import-js/eslint-plugin-import#3004)], thanks \[[@amsardesai](https://github.com/amsardesai)])
-   \[`no-unused-modules`]: Add `ignoreUnusedTypeExports` option (\[[#3011](import-js/eslint-plugin-import#3011)], thanks \[[@silverwind](https://github.com/silverwind)])
-   add support for Flat Config (\[[#3018](import-js/eslint-plugin-import#3018)], thanks \[[@michaelfaith](https://github.com/michaelfaith)])

##### Fixed

-   \[`no-extraneous-dependencies`]: allow wrong path (\[[#3012](import-js/eslint-plugin-import#3012)], thanks \[[@chabb](https://github.com/chabb)])
-   \[`no-cycle`]: use scc algorithm to optimize (\[[#2998](import-js/eslint-plugin-import#2998)], thanks \[[@soryy708](https://github.com/soryy708)])
-   \[`no-duplicates`]: Removing duplicates breaks in TypeScript (\[[#3033](import-js/eslint-plugin-import#3033)], thanks \[[@yesl-kim](https://github.com/yesl-kim)])
-   \[`newline-after-import`]: fix considerComments option when require (\[[#2952](import-js/eslint-plugin-import#2952)], thanks \[[@developer-bandi](https://github.com/developer-bandi)])
-   \[`order`]: do not compare first path segment for relative paths (\[[#2682](import-js/eslint-plugin-import#2682)]) (\[[#2885](import-js/eslint-plugin-import#2885)], thanks \[[@mihkeleidast](https://github.com/mihkeleidast)])

##### Changed

-   \[Docs] `no-extraneous-dependencies`: Make glob pattern description more explicit (\[[#2944](import-js/eslint-plugin-import#2944)], thanks \[[@mulztob](https://github.com/mulztob)])
-   \[`no-unused-modules`]: add console message to help debug \[[#2866](import-js/eslint-plugin-import#2866)]
-   \[Refactor] `ExportMap`: make procedures static instead of monkeypatching exportmap (\[[#2982](import-js/eslint-plugin-import#2982)], thanks \[[@soryy708](https://github.com/soryy708)])
-   \[Refactor] `ExportMap`: separate ExportMap instance from its builder logic (\[[#2985](import-js/eslint-plugin-import#2985)], thanks \[[@soryy708](https://github.com/soryy708)])
-   \[Docs] `order`: Add a quick note on how unbound imports and --fix (\[[#2640](import-js/eslint-plugin-import#2640)], thanks \[[@MinervaBot](https://github.com/minervabot)])
-   \[Tests] appveyor -> GHA (run tests on Windows in both pwsh and WSL + Ubuntu) (\[[#2987](import-js/eslint-plugin-import#2987)], thanks \[[@joeyguerra](https://github.com/joeyguerra)])
-   \[actions] migrate OSX tests to GHA (\[[ljharb#37](ljharb/eslint-plugin-import#37)], thanks \[[@aks-](https://github.com/aks-)])
-   \[Refactor] `exportMapBuilder`: avoid hoisting (\[[#2989](import-js/eslint-plugin-import#2989)], thanks \[[@soryy708](https://github.com/soryy708)])
-   \[Refactor] `ExportMap`: extract "builder" logic to separate files (\[[#2991](import-js/eslint-plugin-import#2991)], thanks \[[@soryy708](https://github.com/soryy708)])
-   \[Docs] \[`order`]: update the description of the `pathGroupsExcludedImportTypes` option (\[[#3036](import-js/eslint-plugin-import#3036)], thanks \[[@liby](https://github.com/liby)])
-   \[readme] Clarify how to install the plugin (\[[#2993](import-js/eslint-plugin-import#2993)], thanks \[[@jwbth](https://github.com/jwbth)])
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants